chore: prepare 0.13.2 release#44
Conversation
Bump version to 0.13.2 and document the OkHttp read timeout fix (disabled default read timeout; requests remain bounded by callTimeout). Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
📝 WalkthroughWalkthroughThis pull request bumps the project version from 0.13.1 to 0.13.2 in pom.xml and adds a corresponding CHANGELOG.md entry describing a fix that disables OkHttp's default read timeout to prevent premature failures on large payloads during long-running scans. ChangesRelease version bump
Estimated code review effort: 1 (Trivial) | ~2 minutes Related Issues: None found in provided context. Related PRs: None found in provided context. Suggested labels: release, documentation Suggested reviewers: None specified. 🐰
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@CHANGELOG.md`:
- Around line 10-13: The changelog entry for 0.13.2 is missing its version
comparison link reference, so add the corresponding [0.13.2] link in the
reference section at the bottom of CHANGELOG.md. Keep the existing entry text
unchanged, and mirror the format used by other version labels in the changelog
so the new reference resolves consistently.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
| ## [0.13.2] - 2026-07-01 | ||
| ### Fixed | ||
| - Disabled OkHttp's default read timeout so long-running scans are bounded only by the overall call timeout, preventing premature failures on large payloads. | ||
|
|
There was a problem hiding this comment.
📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win
Add missing version comparison link for [0.13.2].
The changelog entry content is accurate and matches the implementation in ScanApi.java (disabling readTimeout while keeping callTimeout). However, the [0.13.2] reference link is missing at the bottom of the file. Add it to maintain consistency with existing versions and ensure the reference resolves.
[0.13.1]: https://github.com/scanoss/scanoss.java/compare/v0.13.0...v0.13.1
+[0.13.2]: https://github.com/scanoss/scanoss.java/compare/v0.13.1...v0.13.2📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| ## [0.13.2] - 2026-07-01 | |
| ### Fixed | |
| - Disabled OkHttp's default read timeout so long-running scans are bounded only by the overall call timeout, preventing premature failures on large payloads. | |
| [0.13.1]: https://github.com/scanoss/scanoss.java/compare/v0.13.0...v0.13.1 | |
| [0.13.2]: https://github.com/scanoss/scanoss.java/compare/v0.13.1...v0.13.2 |
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@CHANGELOG.md` around lines 10 - 13, The changelog entry for 0.13.2 is missing
its version comparison link reference, so add the corresponding [0.13.2] link in
the reference section at the bottom of CHANGELOG.md. Keep the existing entry
text unchanged, and mirror the format used by other version labels in the
changelog so the new reference resolves consistently.
|
Superseded by the 0.13.1 release-prep PR. 0.13.1 was never actually tagged/released (latest tag is v0.13.0), so we release the read-timeout fix as part of 0.13.1 instead of bumping to 0.13.2 and leaving a version gap. |
What
Release preparation for 0.13.2.
0.13.1→0.13.2inpom.xmlCHANGELOG.mdunder### FixedWhy
PR #43 (disable OkHttp default read timeout) was merged to
mainwithout a version bump or CHANGELOG entry. Because the current pom version (0.13.1) already has a matching tag (v0.13.1),tools/get_next_version.shaborts and the release pipeline cannot produce a new tag. This bumps to0.13.2so the release can proceed.Release steps (after merge)
main.workflow_dispatch) withrun_for_real = true→ creates and pushes tagv0.13.2.Verification
The underlying fix was verified with a MockWebServer test:
readTimeout=0, callTimeout=60s) completes a 15s slow read successfully.callTimeoutstill bounds the request: with callTimeout=5s the call aborts at ~5s even with read timeout disabled (roof timeout preserved).🤖 Generated with Claude Code
Summary by CodeRabbit
Bug Fixes
Chores